Skip to content

Use Aspire 8.0 workload in the 9.0 SDK #41562

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jul 1, 2024
Merged

Conversation

joperezr
Copy link
Member

cc: @dsplaisted @marcpopMSFT @baronfel

This change is making it so that the 9.0 SDK will have an 8.0 Aspire workload, as opposed to the 9.0 previews that it had before. Aspire decided to not have 9.0 previews and instead only maintain a single stable 8.0 train, so given this decision, this change will make it so the SDK will carry the latest 8.0 workload as opposed to a dead-ended 9.0 preview one.

We have validated that this will still allow users to use the 9.0 SDK to build Aspire apps, even if they are targeting 9.0 TFMs in their individual services. After this change ships, we'll be able to un-list all of the (now dead-ended) 9.0 preview packages for Aspire workloads, in favor of pushing folks to use to the live 8.0 train.

@joperezr joperezr requested a review from marcpopMSFT June 12, 2024 20:43
@ghost ghost added Area-CodeFlow untriaged Request triage from a team member labels Jun 12, 2024
@marcpopMSFT
Copy link
Member

@joperezr any customer who keeps an older preview installed will be broken for aspire, correct? If so, we'll probably want to either breaking change or relnote this so that there's a place to point customers to tell them to uninstall their old sdk. Would it be worth detecting that a 9.0 aspire manifest is installed (any way to do that?) and provide a warning to help customers unwind this?

@joperezr
Copy link
Member Author

Wouldn't installing a newer preview of the SDK replace a previous installation? (and therefore remove the 9.0 manifest)

If so I'm fine with doing a relnote to basically just tell customers that if they want to use Aspire they should just install latest 9.0 SDK

@marcpopMSFT
Copy link
Member

We only replace the earlier version on windows admin installs. Zip installs and mac/linux installs would not replace the prior version. We've definitely run into problems of people having months old previews still installed fwiw.

@marcpopMSFT
Copy link
Member

/azp run sdk-source-build,sdk-unified-build

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@joperezr
Copy link
Member Author

We only replace the earlier version on windows admin installs. Zip installs and mac/linux installs would not replace the prior version. We've definitely run into problems of people having months old previews still installed fwiw.

I see, in those cases I think we can suggest either uninstalling manually or alternatively using rollback files (which is the current workaround we advertise to folks using the 9.0 SDK) to force the use of the 8.0 workload as opposed to newer manifests.

@marcpopMSFT
Copy link
Member

That's a reasonable approach. I just wanted to ensure there was documentation we could point people at. Remerging as the vmr builds were having issues and let's see if that helps.

@joperezr
Copy link
Member Author

@marcpopMSFT trying to understand the source build failures but not sure I'm following what the issue is. Do you have any clue or do you know who can I work with to resolve? I'd really like this to ship in Preview 6 and I know snap is soon.

@marcpopMSFT
Copy link
Member

@marcpopMSFT trying to understand the source build failures but not sure I'm following what the issue is. Do you have any clue or do you know who can I work with to resolve? I'd really like this to ship in Preview 6 and I know snap is soon.

@dotnet/source-build-internal and @dotnet/product-construction for the source build and VMR failures. Looks like a similar failure in both around merged manifests and not sure how that's related to this PR:
/vmr/eng/merge-asset-manifests.proj(21,5): error MSB4018: The "Microsoft.DotNet.UnifiedBuild.Tasks.MergeAssetManifests" task failed unexpectedly.

@mmitche
Copy link
Member

mmitche commented Jun 19, 2024

The error is about how the manifest produced by aspire's publishing doesn't have the same root attributes as the other repos. Because you rolled the aspire sha back to an older 8.0 version, it's highly likely that it's missing some fixes that were made in mian to work with the VMR.

@joperezr
Copy link
Member Author

I see, is there a way to know which fixes are those? Aspire builds with the 8.0 arcade, so if the fixes are done in arcade main then that would explain it.

@mmitche mmitche requested review from a team as code owners June 21, 2024 18:57
@joperezr
Copy link
Member Author

/backport to release/9.0.1xx-preview6

Copy link
Contributor

Started backporting to release/9.0.1xx-preview6: https://github.com/dotnet/sdk/actions/runs/9667085593

@joperezr
Copy link
Member Author

/azp run sdk-unified-build

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@joperezr joperezr enabled auto-merge June 28, 2024 23:20
@joperezr
Copy link
Member Author

Looks like this is ready to go @marcpopMSFT 🙂

@joperezr joperezr merged commit 437db19 into dotnet:main Jul 1, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-CodeFlow untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants